Skip to content

Add optional limit parameter to suggest queries (#182)#193

Merged
fsteeg merged 4 commits intomasterfrom
182-limitParameter
Jul 10, 2025
Merged

Add optional limit parameter to suggest queries (#182)#193
fsteeg merged 4 commits intomasterfrom
182-limitParameter

Conversation

@fsteeg
Copy link
Copy Markdown
Member

@fsteeg fsteeg commented Jun 12, 2025

To resolve the remaining part of #182.

@netlify
Copy link
Copy Markdown

netlify bot commented Jun 12, 2025

Deploy Preview for reconciliation-api-specs ready!

Name Link
🔨 Latest commit 83bac12
🔍 Latest deploy log https://app.netlify.com/projects/reconciliation-api-specs/deploys/686f8345c089bc0008a792e2
😎 Deploy Preview https://deploy-preview-193--reconciliation-api-specs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@saumier
Copy link
Copy Markdown
Contributor

saumier commented Jun 12, 2025

I my opinion, cursor should be renamed offset as a more common and developer friendly term. This would work pair nicely with limit.

Copy link
Copy Markdown
Member

@wetneb wetneb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree that offset matches the description of the field better (but to me the terms aren't interchangeable).

@fsteeg fsteeg merged commit c9a20ba into master Jul 10, 2025
5 checks passed
@fsteeg fsteeg deleted the 182-limitParameter branch July 10, 2025 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants